Skip to content

✅ Use non superuser token and add perftest for objecttype filter#690

Merged
stevenbal merged 1 commit intomasterfrom
chore/perftests-with-non-superuser-token
Oct 27, 2025
Merged

✅ Use non superuser token and add perftest for objecttype filter#690
stevenbal merged 1 commit intomasterfrom
chore/perftests-with-non-superuser-token

Conversation

@stevenbal
Copy link
Collaborator

@stevenbal stevenbal commented Oct 24, 2025

I slightly modified the performance tests in #678, making them use non superuser tokens, which increases the response times in the performance tests. I'm putting it in a separate PR to show that #678 does actually improve performance, compared to main

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.16%. Comparing base (e39112b) to head (de4e751).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #690   +/-   ##
=======================================
  Coverage   83.16%   83.16%           
=======================================
  Files         128      128           
  Lines        2429     2429           
  Branches      193      193           
=======================================
  Hits         2020     2020           
  Misses        369      369           
  Partials       40       40           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Contributor

🐰 Bencher Report

Branchchore/perftests-with-non-superuser-token
Testbedubuntu-latest

🚨 4 Alerts

BenchmarkMeasure
Units
ViewBenchmark Result
(Result Δ%)
Upper Boundary
(Limit %)
performance_test/tests/test_objects_list.py::test_objects_api_list_filter_one_resultLatency
milliseconds (ms)
📈 plot
🚷 threshold
🚨 alert (🔔)
22.15 ms
(+7.41%)Baseline: 20.62 ms
21.65 ms
(102.30%)

performance_test/tests/test_objects_list.py::test_objects_api_list_large_page_size_page_1Latency
milliseconds (ms)
📈 plot
🚷 threshold
🚨 alert (🔔)
319.19 ms
(+17.22%)Baseline: 272.30 ms
285.91 ms
(111.64%)

performance_test/tests/test_objects_list.py::test_objects_api_list_large_page_size_page_5Latency
milliseconds (ms)
📈 plot
🚷 threshold
🚨 alert (🔔)
326.38 ms
(+17.41%)Baseline: 277.97 ms
291.87 ms
(111.82%)

performance_test/tests/test_objects_list.py::test_objects_api_list_small_page_size_page_20Latency
milliseconds (ms)
📈 plot
🚷 threshold
🚨 alert (🔔)
183.74 ms
(+32.76%)Baseline: 138.40 ms
145.32 ms
(126.44%)

Click to view all benchmark results
BenchmarkLatencyBenchmark Result
milliseconds (ms)
(Result Δ%)
Upper Boundary
milliseconds (ms)
(Limit %)
performance_test/tests/test_objects_list.py::test_objects_api_list_filter_by_object_type📈 view plot
🚷 view threshold
186.13 ms
(0.00%)Baseline: 186.13 ms
195.44 ms
(95.24%)
performance_test/tests/test_objects_list.py::test_objects_api_list_filter_one_result📈 view plot
🚷 view threshold
🚨 view alert (🔔)
22.15 ms
(+7.41%)Baseline: 20.62 ms
21.65 ms
(102.30%)

performance_test/tests/test_objects_list.py::test_objects_api_list_large_page_size_page_1📈 view plot
🚷 view threshold
🚨 view alert (🔔)
319.19 ms
(+17.22%)Baseline: 272.30 ms
285.91 ms
(111.64%)

performance_test/tests/test_objects_list.py::test_objects_api_list_large_page_size_page_5📈 view plot
🚷 view threshold
🚨 view alert (🔔)
326.38 ms
(+17.41%)Baseline: 277.97 ms
291.87 ms
(111.82%)

performance_test/tests/test_objects_list.py::test_objects_api_list_small_page_size_page_20📈 view plot
🚷 view threshold
🚨 view alert (🔔)
183.74 ms
(+32.76%)Baseline: 138.40 ms
145.32 ms
(126.44%)

🐰 View full continuous benchmarking report in Bencher

@stevenbal stevenbal merged commit 535ca32 into master Oct 27, 2025
27 checks passed
@stevenbal stevenbal deleted the chore/perftests-with-non-superuser-token branch October 27, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants